Skip to content

[JENKINS-30141] Option to choose if depth is sticky#134

Open
vvv444 wants to merge 4 commits into
jenkinsci:masterfrom
vvv444:master
Open

[JENKINS-30141] Option to choose if depth is sticky#134
vvv444 wants to merge 4 commits into
jenkinsci:masterfrom
vvv444:master

Conversation

@vvv444

@vvv444 vvv444 commented Aug 26, 2015

Copy link
Copy Markdown

https://issues.jenkins-ci.org/browse/JENKINS-30141

User can choose the desired behavior now (like --depth and --set-depth flags in svn)

User can choose the desired behavior now (like --depth and --set-depth flags in svn)
@jenkinsadmin

Copy link
Copy Markdown
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@recena recena changed the title Option to choose if depth is sticky [JENKINS-30141] Option to choose if depth is sticky Aug 26, 2015

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

binary compatibility issue. It's better to keep and deprecate the original constructor

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! I'm quite new to Java :-)
I've pushed a fix commit to my repository. Now, how is it customary to
proceed in such cases? Shall I open a new pull request?

On Wed, Sep 9, 2015 at 12:32 PM, Oleg Nenashev notifications@github.com
wrote:

In src/main/java/hudson/scm/SubversionSCM.java
#134 (comment)
:

     }

     @DataBoundConstructor
  •    public ModuleLocation(String remote, String credentialsId, String local, String depthOption, boolean ignoreExternalsOption) {
    
  •    public ModuleLocation(String remote, String credentialsId, String local, String depthOption, boolean stickyDepth, boolean ignoreExternalsOption) {
    

binary compatibility issue. It's better to keep and deprecate the original
constructor


Reply to this email directly or view it on GitHub
https://github.com/jenkinsci/subversion-plugin/pull/134/files#r39023278.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also create a help file for this new option?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. Done :-)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vvv444 Please, use <code> instead of <tt>. I'll try to merge this PR soon.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@vvv444

vvv444 commented Oct 12, 2015

Copy link
Copy Markdown
Author

Please note the check failed due to some timeout problem. Not because of my code. How can I rerun it?

@recena

recena commented Oct 13, 2015

Copy link
Copy Markdown
Contributor

@vvv444 close and open the PR.

@vvv444 vvv444 closed this Oct 13, 2015
@vvv444 vvv444 reopened this Oct 13, 2015
@vvv444

vvv444 commented Feb 3, 2016

Copy link
Copy Markdown
Author

@recena @oleg-nenashev Hey guys, any reason why this wasn't integrated?

@recena

recena commented Feb 3, 2016

Copy link
Copy Markdown
Contributor

@vvv444 Yes, I'm avoiding to include new features until we improve our test coverage and solve bugs related to svn:externals. I'm sorry for that.

@jglick

jglick commented Jun 16, 2017

Copy link
Copy Markdown
Member

I would also prefer to avoid new features unless they are obviously self-contained like SCMBrowsers.

@rshah88

rshah88 commented Aug 28, 2017

Copy link
Copy Markdown

Hey guys, any ETA on when will this feature be integrated ?

@oleg-nenashev

Copy link
Copy Markdown
Member

Somebody need to resolve the merge conflict

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants